Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't try to preserve owner/permissions when extracting tarball #123

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wjt
Copy link
Member

@wjt wjt commented Jun 3, 2020

When run as root, tar defaults to --same-owner --same-permissions.
This is reported to cause many errors:

tar: .dropbox-dist/VERSION: Cannot change ownership to uid 1000, gid 1000: Operation not permitted
...
tar: Exiting with failure status due to previous errors

While this does not actually cause the apply_extra script to fail, it is
ugly!

Fixes #122

When run as root, `tar` defaults to `--same-owner --same-permissions`.
This is reported to cause many errors:

    tar: .dropbox-dist/VERSION: Cannot change ownership to uid 1000, gid 1000: Operation not permitted
    ...
    tar: Exiting with failure status due to previous errors

While this does not actually cause the apply_extra script to fail, it is
ugly!

Fixes #122
@flathubbot
Copy link
Contributor

Started test build 21007

@flathubbot
Copy link
Contributor

Build 21007 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/20157/com.dropbox.Client.flatpakref

@wjt
Copy link
Member Author

wjt commented Jun 3, 2020

No dice, nothing appears in /app/extra with this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update on CentOS 7 results in many tar permission errors
2 participants